Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add typed dict type checking #4340

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

adhami3310
Copy link
Member

No description provided.

@adhami3310
Copy link
Member Author

So this also included some changes in the check of var values being of the correct type. This has uncovered some issues in reflex-web. I will make a PR there and update this one once that has merged. @masenf

@adhami3310
Copy link
Member Author

blocked on #1079

@benedikt-bartscher
Copy link
Contributor

blocked on #1079

Links to wrong repo
reflex-dev/reflex-web#1079

@adhami3310
Copy link
Member Author

adhami3310 commented Nov 11, 2024

Links to wrong repo

ooops, defn not blocked by something that merged years ago 😅

@adhami3310
Copy link
Member Author

is ready for merge, although we should really fix check_latest_node

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants